Skip to content
This repository was archived by the owner on Mar 29, 2021. It is now read-only.

Idrinth#308

Merged
w20k merged 6 commits intomasterfrom
idrinth
Apr 8, 2017
Merged

Idrinth#308
w20k merged 6 commits intomasterfrom
idrinth

Conversation

@Idrinth
Copy link
Copy Markdown
Member

@Idrinth Idrinth commented Apr 4, 2017

No description provided.

Comment thread src/mods/tier.js
idrinth.tier.list[listKey].loot.hasOwnProperty ( difficulty ) &&
idrinth.tier.list[listKey].loot[difficulty].hasOwnProperty ( ic ) &&
idrinth.tier.list[listKey].loot[difficulty][ic]
) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[jshint]

  • The array literal notation [] is preferable.

@Idrinth
Copy link
Copy Markdown
Member Author

Idrinth commented Apr 8, 2017

@Ardenian @w20k anyone around to review this?

@w20k
Copy link
Copy Markdown
Collaborator

w20k commented Apr 8, 2017

Will take a look ;(

@w20k
Copy link
Copy Markdown
Collaborator

w20k commented Apr 8, 2017

Wrong emoji :))

@w20k
Copy link
Copy Markdown
Collaborator

w20k commented Apr 8, 2017

@w20k w20k merged commit b9949c2 into master Apr 8, 2017
@Idrinth Idrinth deleted the idrinth branch April 8, 2017 12:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants